-
Notifications
You must be signed in to change notification settings - Fork 9.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some issues reported by LGTM #2000
Conversation
This fixes a warning from LGTM: No matching copy assignment operator in class LineHypothesis. It is good practice to match a copy constructor with a copy assignment operator. Signed-off-by: Stefan Weil <sw@weilnetz.de>
This fixes a warning from LGTM: No matching copy assignment operator in class ParamsTrainingHypothesis. It is good practice to match a copy constructor with a copy assignment operator. Use also a simpler expression for the size of features. Signed-off-by: Stefan Weil <sw@weilnetz.de>
This fixes a warning from LGTM: No matching copy assignment operator in class BLOB_CHOICE. It is good practice to match a copy constructor with a copy assignment operator. Signed-off-by: Stefan Weil <sw@weilnetz.de>
It does not need an implementation as it is currently not used. This fixes a warning from LGTM: No matching copy constructor in class ROW. It is good practice to match a copy assignment operator with a copy constructor. Signed-off-by: Stefan Weil <sw@weilnetz.de>
It does not need an implementation as it is currently not used. This fixes a warning from LGTM: No matching copy constructor in class C_OUTLINE_FRAG. It is good practice to match a copy assignment operator with a copy constructor. Signed-off-by: Stefan Weil <sw@weilnetz.de>
It does not need an implementation as it is currently not used. This fixes a warning from LGTM: No matching copy assignment operator in class BlamerBundle. It is good practice to match a copy constructor with a copy assignment operator. Signed-off-by: Stefan Weil <sw@weilnetz.de>
This pull request fixes 6 alerts when merging f0c9b75 into 9c2d1aa - view on LGTM.com fixed alerts:
Comment posted by LGTM.com |
Thank you for the link. It's always nice to learn. So there is a more elegant C++11 way for the last three commits. I suggest to apply the pull request as it is and would use the C++11 form later (maybe after 4.0.0), as there is more Tesseract code which can be updated for C++11. |
Okay. Apart from |
Just to celebrate #2000 ... |
No description provided.